Skip to content

update_checkout: add swift-subprocess to the checkout set #83104

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

update_checkout: add swift-subprocess to the checkout set #83104

wants to merge 2 commits into from

Conversation

compnerd
Copy link
Member

Add swift-subprocess to the checkout set so that we can start investigating building that for integration into swift-build.

@compnerd
update_checkout: add swift-subprocess to the checkout set
5bdef44 
Add swift-subprocess to the checkout set so that we can start
investigating building that for integration into swift-build.
@compnerd compnerd requested a review from shahmishal as a code owner July 16, 2025 17:47
@compnerd compnerd requested a review from jakepetroules July 16, 2025 17:47
@compnerd
Copy link
Member Author

@swift-ci please smoke test

@shahmishal
Copy link
Member

Can we use a tag instead of main branch? @itingliu is this something you can help us with?

@iCharlesHu
Copy link
Contributor

iCharlesHu commented Jul 19, 2025 •
edited
Loading

There are still a few issues for non Darwin platforms I need to resolve before we can tag 0.0.1. I'll report back as soon as we do.

@itingliu
Copy link

There are still a few issues for non Darwin platforms I need to resolve before we can tag 0.0.1. I'll report back as soon as we do.

As we just agreed, we will tag it with a name that's clear if's only for development purposes and not a release tag.

@iCharlesHu
Copy link
Contributor

@itingliu and I had a discussion we decided to release development-snapshot as needed, similar to Swift's snapshots.
https://github.com/swiftlang/swift-subprocess/tags

@compnerd
Copy link
Member Author

I'm not sure what the change needed here is then. Should this just point to dev?

itingliu
itingliu reviewed Jul 21, 2025
View reviewed changes
@compnerd
Copy link
Member Author

@swift-ci please smoke test

@compnerd
Copy link
Member Author

@shahmishal - okay to merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itingliu itingliu itingliu left review comments

@jakepetroules jakepetroules jakepetroules approved these changes

@shahmishal shahmishal Awaiting requested review from shahmishal shahmishal is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@compnerd @shahmishal @iCharlesHu @itingliu @jakepetroules